From: Andrew Cooper Date: Thu, 19 Nov 2015 12:43:52 +0000 (+0000) Subject: tools/libxl: Drop dead code following calls to libxl__exec() X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~2213 X-Git-Url: https://dgit.raspbian.org/%22http:/www.example.com/cgi/%22https://%22%22/%22http:/www.example.com/cgi/%22https:/%22%22?a=commitdiff_plain;h=2dd7eaf2e66eda7849c6b1a9f994aa391250ab7f;p=xen.git tools/libxl: Drop dead code following calls to libxl__exec() libxl__exec() doesn't ever return. Inform the compiler of this, and remove all dead code. No functional change. Signed-off-by: Andrew Cooper Acked-by: Ian Campbell --- diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index 854e957c08..6ad9e13ffc 100644 --- a/tools/libxl/libxl.c +++ b/tools/libxl/libxl.c @@ -1983,7 +1983,6 @@ int libxl_vncviewer_exec(libxl_ctx *ctx, uint32_t domid, int autopass) } libxl__exec(gc, autopass_fd, -1, -1, args[0], args, NULL); - abort(); x_fail: GC_FREE; diff --git a/tools/libxl/libxl_aoutils.c b/tools/libxl/libxl_aoutils.c index d5fbc4d862..9e493cd487 100644 --- a/tools/libxl/libxl_aoutils.c +++ b/tools/libxl/libxl_aoutils.c @@ -603,8 +603,6 @@ int libxl__async_exec_start(libxl__async_exec_state *aes) /* child */ libxl__exec(gc, aes->stdfds[0], aes->stdfds[1], aes->stdfds[2], args[0], args, aes->env); - /* notreached */ - abort(); } return 0; diff --git a/tools/libxl/libxl_bootloader.c b/tools/libxl/libxl_bootloader.c index 95dde986f2..9caf212ab0 100644 --- a/tools/libxl/libxl_bootloader.c +++ b/tools/libxl/libxl_bootloader.c @@ -556,7 +556,6 @@ static void bootloader_gotptys(libxl__egc *egc, libxl__openpty_state *op) r = login_tty(libxl__carefd_fd(bl->ptys[0].slave)); if (r) { LOGE(ERROR, "login_tty failed"); exit(-1); } libxl__exec(gc, -1, -1, -1, bl->args[0], (char **) bl->args, env); - exit(-1); } /* parent */ diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h index 590870a2f7..58d07cd4f7 100644 --- a/tools/libxl/libxl_internal.h +++ b/tools/libxl/libxl_internal.h @@ -1578,7 +1578,7 @@ _hidden int libxl__xenstore_child_wait_deprecated(libxl__gc *gc, */ _hidden void libxl__exec(libxl__gc *gc, int stdinfd, int stdoutfd, int stderrfd, const char *arg0, char *const args[], - char *const env[]); + char *const env[]) __attribute__((noreturn)); /* from xl_create */